Skip to content

[meta] Force enums with strict flags to have zero enum defined#2264

Merged
tjchadaga merged 1 commit intoopencomputeproject:masterfrom
kcudnik:flagzero
Mar 26, 2026
Merged

[meta] Force enums with strict flags to have zero enum defined#2264
tjchadaga merged 1 commit intoopencomputeproject:masterfrom
kcudnik:flagzero

Conversation

@kcudnik
Copy link
Copy Markdown
Collaborator

@kcudnik kcudnik commented Mar 20, 2026

This could be handy for serialization purposes if no flags are defined in serialized enum. But actual purpose of having zero value could have no meaning (for exaple stats mode)

This could be handy for serialization purposes if no flags
are defined in serialized enum. But actual purpose of having
zero value could have no meaning (for exaple stats mode)
@kcudnik
Copy link
Copy Markdown
Collaborator Author

kcudnik commented Mar 20, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@tjchadaga tjchadaga merged commit 59a504f into opencomputeproject:master Mar 26, 2026
3 checks passed
@kcudnik kcudnik deleted the flagzero branch March 26, 2026 20:12
tjchadaga pushed a commit that referenced this pull request Mar 31, 2026
This could be handy for serialization purposes if no flags
are defined in serialized enum. But actual purpose of having
zero value could have no meaning (for exaple stats mode)
cursor Bot pushed a commit to yuriilisovskyi/SAI that referenced this pull request Apr 7, 2026
…omputeproject#2264)

This could be handy for serialization purposes if no flags
are defined in serialized enum. But actual purpose of having
zero value could have no meaning (for exaple stats mode)
rpmarvell pushed a commit to rpmarvell/SAI that referenced this pull request Apr 8, 2026
…omputeproject#2264)

This could be handy for serialization purposes if no flags
are defined in serialized enum. But actual purpose of having
zero value could have no meaning (for exaple stats mode)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants